docs(linux): AM64X: Add ICSSG PRP Mode Docs#348
Merged
Conversation
hm-ti
requested review from
StaticRocket,
VeeruPrudhvi,
aniket-l,
cshilwant,
praneethbajjuri and
uditkumarti
as code owners
github-actions
Bot
assigned
devarsht,
praneethbajjuri,
r-vignesh,
Siddharth-Vadapalli-at-TI and
uditkumarti
cshilwant
requested changes
Jul 9, 2025
|
New warnings found with rstcheck: |
danish-ti
suggested changes
Jul 10, 2025
uditkumarti
previously approved these changes
Jul 10, 2025
|
New warnings found with rstcheck: |
cshilwant
requested changes
Jul 10, 2025
StaticRocket
requested changes
Jul 10, 2025
StaticRocket
left a comment
StaticRocket
left a comment
Member
There was a problem hiding this comment.
Please address the vale comments and fix the csv table currently generating errors.
StaticRocket
requested changes
Jul 11, 2025
StaticRocket
left a comment
StaticRocket
left a comment
Member
There was a problem hiding this comment.
This probably should have been 3 commits with more reasonable information than just docs(linux): AM64X: Update docs, and that's without commenting on https://github.com/TexasInstruments/processor-sdk-doc/blob/master/CONTRIBUTING.md#commit-formatting
danish-ti
approved these changes
Jul 14, 2025
|
New issues found with check_toc_txt.py: |
StaticRocket
requested changes
Jul 14, 2025
StaticRocket
left a comment
StaticRocket
left a comment
Member
There was a problem hiding this comment.
This should use the feat tag, not the docs tag
|
New issues found with check_toc_txt.py: |
|
New warnings found with rstcheck: |
Member
|
The commit tags need to be updated. You fixed the first one, but the second one is now wrong. I'd like to suggest: There are also some valid vale warnings (usage of linux instead of Linux, etc.) |
hm-ti
force-pushed
the
master
branch
2 times, most recently
from
1ae5cec to
681cc02
Compare
- Adds documentation on ICSSG PRP offload mode supported by prueth driver Signed-off-by: Himanshu Mittal <h-mittal1@ti.com>
- Issue Tracker Update containing details of ICSSG firmware
related bugs resolved and bugs open.
- Adds new PRP feature in What's New section
Signed-off-by: Himanshu Mittal <h-mittal1@ti.com>
StaticRocket
approved these changes
Jul 16, 2025
cshilwant
approved these changes
Jul 21, 2025
jeevantelukula
approved these changes
Jul 21, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
10 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds documentation on PRP offload mode supported by ICSSG